Skip to content

fix(skills): honor offline TTS with expired HeyGen auth#2330

Closed
miguel-heygen wants to merge 1 commit into
mainfrom
fix/audio-expired-auth-provider
Closed

fix(skills): honor offline TTS with expired HeyGen auth#2330
miguel-heygen wants to merge 1 commit into
mainfrom
fix/audio-expired-auth-provider

Conversation

@miguel-heygen

Copy link
Copy Markdown
Collaborator

What

  • Treat expired HeyGen OAuth as unavailable instead of eagerly throwing before provider selection.
  • Let the faceless-explainer adapter honor --provider and HF_TTS_PROVIDER, with the CLI flag taking precedence.
  • Document explicit provider selection in the faceless workflow.

Why

A user explicitly selecting local Kokoro still hit heygenAuthHeaders() during shared audio-engine startup. An expired OAuth credential therefore aborted the run before the engine could honor the offline provider.

Reproduction

Confirmed against current main with an expired OAuth credentials file and an audio_request.json selecting Kokoro: the shared engine exited at startup with HeyGen OAuth token expired. The same reproduction exits successfully after this change.

Tests

  • Expired OAuth is not reported as an available HeyGen TTS provider.
  • Explicit Kokoro bypasses expired HeyGen auth in the shared engine.
  • Faceless adapter covers CLI flag passthrough, env fallback, and flag precedence.
  • bun run test:skills (234 passed)
  • bun run lint:skills
  • targeted oxfmt --check
  • git diff --check

@miguel-heygen

Copy link
Copy Markdown
Collaborator Author

Consolidated with the related TTS provider and error-handling fixes in #2407. The still-relevant behavior from the retired skill path is ported into media-use with regression coverage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant